Skip to content

fix: split typst list items into separate sentences#206

Open
saschabuehrle wants to merge 1 commit into
jeertmans:mainfrom
saschabuehrle:fix/issue-205-typst-list-separator
Open

fix: split typst list items into separate sentences#206
saschabuehrle wants to merge 1 commit into
jeertmans:mainfrom
saschabuehrle:fix/issue-205-typst-list-separator

Conversation

@saschabuehrle

Copy link
Copy Markdown

Fixes #205

Typst list and enum items were emitted back-to-back in the parsed text, so LanguageTool treated the next item as a continuation of the previous sentence.

This inserts a newline annotation after each ListItem/EnumItem so every list entry starts as its own sentence.

Greetings, saschabuehrle

@tschogge

tschogge commented Apr 2, 2026

Copy link
Copy Markdown

Thank you very much for this fix

@jeertmans

Copy link
Copy Markdown
Owner

Hi @saschabuehrle, thanks for your contribution! Could you add one or more tests in https://github.com/jeertmans/languagetool-rust/blob/main/tests/sample_files/example.typ and sync the snapshot files to assert that the list items are checked as expected? Running the tests will normally re-generate those snapshots files (see https://docs.rs/insta/latest/insta/).

@jeertmans jeertmans added bug Something isn't working cli Related to the CLI labels Apr 4, 2026
@codspeed-hq

codspeed-hq Bot commented Apr 4, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 6 untouched benchmarks


Comparing saschabuehrle:fix/issue-205-typst-list-separator (61ad7e7) with main (8e46cd9)

Open in CodSpeed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cli Related to the CLI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: List in Typst not recognized as a sentence

3 participants